Claude finished @mbevc1's task in 1m 37s —— View job

PR Review: Remove visibility flag and warn on deprecated flow endpoint

• Run static analysis (build, vet, mod tidy)
• Read and review changed files
• Post review feedback

Static Analysis

All checks pass: go build, go vet, go mod tidy — no issues.

Review Summary

Clean, well-scoped PR. The --visibility flag is removed from create flow, the Visibility field is dropped from FlowPayload, and a deprecation warning is emitted when the legacy PUT /api/v2/flows/{org} endpoint is still used (i.e. neither --template-file nor --use-empty-template is provided). Tests are updated to match.

Findings:

1. Good: CreateFlow helper now uses --use-empty-template (testHelpers.go:345) — this means other test suites that call CreateFlow() no longer hit the deprecated endpoint, avoiding noisy warnings in test output. Thoughtful change.

2. Minor: goldenRegex anchor removed — In the "fails when name is invalid" test case (createFlow_test.go:40), the ^ anchor was dropped from "^Error: .*foo bar" → "Error: .*foo bar". This is correct because the deprecation warning now precedes the error text, so the match can no longer start at the beginning of the output. Good catch.

3. Note: visibility still appears in read-only commands — getFlow.go:110 and listFlows.go:81,90 still display a visibility column from the API response. This is fine — the API still returns it; this PR only removes the ability to set it via create flow. Just noting for awareness in case follow-up cleanup is planned.

4. No breaking change for existing users — Users who pass --visibility to create flow will now get an "unknown flag" error. Since the flag was only meaningful for the deprecated endpoint, this is the correct behavior, but worth mentioning in release notes.

Overall: Looks good to merge. The change is minimal, correct, and well-tested. No bugs, security issues, or performance concerns.